Skip to content

fix(deps): downgrade Microsoft.CodeAnalysis packages to 5.0.0#316

Merged
j-d-ha merged 7 commits into
mainfrom
bug/downgrade-microsoft-code-analysis
Mar 23, 2026
Merged

fix(deps): downgrade Microsoft.CodeAnalysis packages to 5.0.0#316
j-d-ha merged 7 commits into
mainfrom
bug/downgrade-microsoft-code-analysis

Conversation

@j-d-ha

@j-d-ha j-d-ha commented Mar 23, 2026

Copy link
Copy Markdown
Collaborator

🚀 Pull Request

📋 Summary

  • Downgrades all Microsoft.CodeAnalysis* packages from 5.3.0 to 5.0.0.
  • Microsoft.CodeAnalysis.CSharp 5.3.0 introduced a regression (dotnet/roslyn#82780) where source-generator-produced interceptors are silently not applied by the compiler — the interceptor file is generated successfully but the original method executes instead of the intercepted one.
  • This project relies on interceptors for its core functionality, so the broken compiler behavior in 5.3.0 would cause generated Lambda handler wiring to not work at runtime.
  • Pinning back to 5.0.0 (the last known-good version) restores correct interceptor behaviour while the upstream Roslyn bug is tracked and fixed.

✅ Checklist

  • My changes build cleanly
  • I've added/updated relevant tests
  • I've added/updated documentation or README
  • I've followed the coding style for this project
  • I've tested the changes locally (if applicable)

🧪 Related Issues or PRs


💬 Notes for Reviewers

This is a temporary pin. Once the Roslyn team ships a fix, we should upgrade again. The only file changed is Directory.Packages.props — four package version entries.

- Adjusted versions of Microsoft.CodeAnalysis, Analyzers, Common, and CSharp from 5.3.0 to 5.0.0
- Ensures compatibility with existing project dependencies
@github-actions github-actions Bot added the type: fix Bug fix label Mar 23, 2026
Comment thread Directory.Packages.props
j-d-ha added 2 commits March 23, 2026 10:58
- Updated System.Text.Json version to [8.0.6, 9.0.0) to ensure compatibility and future-proofing
- Applies only to netstandard2.0 target framework
…lysis' into bug/downgrade-microsoft-code-analysis
@j-d-ha
j-d-ha marked this pull request as draft March 23, 2026 15:01
j-d-ha added 2 commits March 23, 2026 11:04
- Updated Microsoft.CodeAnalysis.Analyzers to version 5.3.0
- Adjusted Microsoft.CodeAnalysis, Common, and CSharp to use version range [5.0.0]
- Ensures compatibility with project dependencies and resolves versioning issues
- Removed Microsoft.CodeAnalysis.Common from project and central package configs
- Consolidated usage under Microsoft.CodeAnalysis.CSharp where applicable
- Simplifies dependency tree and avoids redundant package references
@j-d-ha
j-d-ha marked this pull request as ready for review March 23, 2026 15:07
@codecov

codecov Bot commented Mar 23, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #316      +/-   ##
==========================================
+ Coverage   87.34%   87.38%   +0.03%     
==========================================
  Files         113      113              
  Lines        2814     2814              
  Branches      351      351              
==========================================
+ Hits         2458     2459       +1     
+ Misses        255      254       -1     
  Partials      101      101              

see 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 457fdaf...8966a45. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@ncipollina ncipollina left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM!

@sonarqubecloud

Copy link
Copy Markdown

@j-d-ha
j-d-ha merged commit 00a8049 into main Mar 23, 2026
8 checks passed
@j-d-ha
j-d-ha deleted the bug/downgrade-microsoft-code-analysis branch March 23, 2026 15:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: fix Bug fix

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants